]> dgit.raspbian.org Git - gnome-shell.git/commit
switcherPopup: Use local variable for index in scrolling functions
authorJonas Dreßler <verdre@v0yd.nl>
Wed, 25 Apr 2018 21:24:05 +0000 (23:24 +0200)
committerFlorian Müllner <florian.muellner@gmail.com>
Wed, 26 Feb 2020 22:46:51 +0000 (22:46 +0000)
commit172d21cf506e5fb6f49d4bad344323e289365442
treea32e94b6ea79c1b5a592454c2ac8050d18d739fa
parent5c8f3a65f79a2e5a10c388ce3fc3c61441e7084e
switcherPopup: Use local variable for index in scrolling functions

Make sure the index that's being scrolled to doesn't change while the
scrolling animation is running by using an argument instead of the
this._highlighed class scope variable.

This fixes wrongly shown arrows when selecting new elements faster than
the scrolling animation takes for one index. The check run to disable
the arrow might be checking against a newer index than the one at the
start of the animation which results in the arrow not getting hidden
even if no more scrolling is possible.

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/167
js/ui/switcherPopup.js